Skip to content

Fix typos#2983

Open
abhro wants to merge 1 commit into
EnzymeAD:mainfrom
abhro:typos
Open

Fix typos#2983
abhro wants to merge 1 commit into
EnzymeAD:mainfrom
abhro:typos

Conversation

@abhro

@abhro abhro commented Feb 25, 2026

Copy link
Copy Markdown
Contributor

Housekeeping edits: Fix up typos.

Update: also ran runic on the changed files after CI workflow

@abhro abhro changed the title Fix Typos Fix typos Feb 25, 2026
@github-actions

github-actions Bot commented Feb 25, 2026

Copy link
Copy Markdown
Contributor

Your PR requires formatting changes to meet the project's style guidelines.
Please consider running Runic (git runic main) to apply these changes.

Click here to view the suggested changes.
diff --git a/src/compiler.jl b/src/compiler.jl
index 8f016715..a16d2e43 100644
--- a/src/compiler.jl
+++ b/src/compiler.jl
@@ -3434,7 +3434,7 @@ function create_abi_wrapper(
 	    # Enzyme expects, arg, [w x darg], root, droot
 	    # Julia expects   arg, root, darg, droot
 	    # We already pushed arg
-	    # now params[i] refers to root
+            # now params[i] refers to root
 	    darg = nothing
 	    root = nothing
 	    droot = nothing
@@ -6813,7 +6813,7 @@ const DumpLLVMCall = Ref(false)
                     tape = callparams[end-1]
                 end
 		if value_type(tape) != llty
-		   throw(AssertionError("Mismatched Tape type, expected $(string(value_type(tape))) found $(string(llty)) from $TapeType arg_roots=$arg_roots"))
+                    throw(AssertionError("Mismatched Tape type, expected $(string(value_type(tape))) found $(string(llty)) from $TapeType arg_roots=$arg_roots"))
 		end
             end
         end
diff --git a/src/utils.jl b/src/utils.jl
index 4f9662c3..52161a1e 100644
--- a/src/utils.jl
+++ b/src/utils.jl
@@ -145,8 +145,8 @@ function unsafe_to_llvm(B::LLVM.IRBuilder, @nospecialize(val); insert_name_if_no
 	    if !inactive && world isa UInt
                 legal, jTy, byref = Compiler.abs_typeof(gv, true)
                 if legal
-                    current_bb = position(B)
-                    fn = LLVM.parent(current_bb)
+                current_bb = position(B)
+                fn = LLVM.parent(current_bb)
 		    state = Enzyme.Compiler.active_reg(jTy, world)
 		    inactive = state == Enzyme.Compiler.AnyState ||state == Enzyme.Compiler.ActiveState
                 end

Comment thread src/compiler/orcv2.jl
@abhro abhro marked this pull request as draft March 4, 2026 03:22
@abhro abhro force-pushed the typos branch 2 times, most recently from cc05f21 to 5dedf92 Compare March 18, 2026 16:39
@abhro abhro marked this pull request as ready for review March 18, 2026 16:41
@codecov

codecov Bot commented Mar 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.10%. Comparing base (fb41cc9) to head (1275b9d).
⚠️ Report is 11 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (fb41cc9) and HEAD (1275b9d). Click for more details.

HEAD has 16 uploads less than BASE
Flag BASE (fb41cc9) HEAD (1275b9d)
23 7
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2983       +/-   ##
===========================================
- Coverage   69.86%   51.10%   -18.76%     
===========================================
  Files          66       13       -53     
  Lines       21718     1262    -20456     
===========================================
- Hits        15174      645    -14529     
+ Misses       6544      617     -5927     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants