[Mlir-commits] [mlir] 1234fc3 - [mlir][LLVM] Fix aliasing in intrinsic base class

[Christian Ulmann]

Author: Christian Ulmann
Date: 2023-05-24T15:02:23Z
New Revision: 1234fc3021503700cf3cb7adcec0423b2fb76373

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

LOG: [mlir][LLVM] Fix aliasing in intrinsic base class

This commit fixes a bug in the intrinsic base class that caused the
declaration of alias analysis attributes under a wrong condition.

Added: 
    

Modified: 
    mlir/include/mlir/Dialect/LLVMIR/LLVMOpBase.td

Removed: 
    


################################################################################
diff  --git a/mlir/include/mlir/Dialect/LLVMIR/LLVMOpBase.td b/mlir/include/mlir/Dialect/LLVMIR/LLVMOpBase.td
index 96ebf42dec328..00a3b80ecf69a 100644
--- a/mlir/include/mlir/Dialect/LLVMIR/LLVMOpBase.td
+++ b/mlir/include/mlir/Dialect/LLVMIR/LLVMOpBase.td
@@ -309,7 +309,7 @@ class LLVM_IntrOpBase<Dialect dialect, string opName, string enumName,
         !if(!gt(requiresAccessGroup, 0),
             (ins OptionalAttr<SymbolRefArrayAttr>:$access_groups),
             (ins )),
-        !if(!gt(requiresAccessGroup, 0),
+        !if(!gt(requiresAliasAnalysis, 0),
             (ins OptionalAttr<SymbolRefArrayAttr>:$alias_scopes,
                  OptionalAttr<SymbolRefArrayAttr>:$noalias_scopes,
                  OptionalAttr<SymbolRefArrayAttr>:$tbaa),