[llvm] d509972 - [SPIRVPrepareFunctions] Report change when removing lifetime intrinsics

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 22 08:04:28 PDT 2025


Author: Nikita Popov
Date: 2025-07-22T17:04:20+02:00
New Revision: d5099722e8ae0972a4a95927cb8ed415eda17298

URL: https://github.com/llvm/llvm-project/commit/d5099722e8ae0972a4a95927cb8ed415eda17298
DIFF: https://github.com/llvm/llvm-project/commit/d5099722e8ae0972a4a95927cb8ed415eda17298.diff

LOG: [SPIRVPrepareFunctions] Report change when removing lifetime intrinsics

Should hopefully fix EXPENSIVE_CHECKS build.

Added: 
    

Modified: 
    llvm/lib/Target/SPIRV/SPIRVPrepareFunctions.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/SPIRV/SPIRVPrepareFunctions.cpp b/llvm/lib/Target/SPIRV/SPIRVPrepareFunctions.cpp
index 6766bd866cd2e..595424b999439 100644
--- a/llvm/lib/Target/SPIRV/SPIRVPrepareFunctions.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVPrepareFunctions.cpp
@@ -410,6 +410,7 @@ bool SPIRVPrepareFunctions::substituteIntrinsicCalls(Function *F) {
               II, Intrinsic::SPVIntrinsics::spv_lifetime_start, {1});
         } else {
           II->eraseFromParent();
+          Changed = true;
         }
         break;
       case Intrinsic::lifetime_end:
@@ -418,6 +419,7 @@ bool SPIRVPrepareFunctions::substituteIntrinsicCalls(Function *F) {
               II, Intrinsic::SPVIntrinsics::spv_lifetime_end, {1});
         } else {
           II->eraseFromParent();
+          Changed = true;
         }
         break;
       case Intrinsic::ptr_annotation:


        


More information about the llvm-commits mailing list