[all-commits] [llvm/llvm-project] 8ddfb6: [flang] Fix MASKR/MASKL lowering for INTEGER(16) (...

[jeanPerier via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8ddfb66903969224ebd4e10c1461d2be323f4798
      https://github.com/llvm/llvm-project/commit/8ddfb66903969224ebd4e10c1461d2be323f4798
  Author: jeanPerier <jperier at nvidia.com>
  Date:   2024-04-08 (Mon, 08 Apr 2024)

  Changed paths:
    M flang/include/flang/Optimizer/Builder/FIRBuilder.h
    M flang/lib/Lower/ConvertVariable.cpp
    M flang/lib/Lower/DirectivesCommon.h
    M flang/lib/Optimizer/Builder/FIRBuilder.cpp
    M flang/lib/Optimizer/Builder/IntrinsicCall.cpp
    M flang/lib/Optimizer/Builder/PPCIntrinsicCall.cpp
    M flang/test/Lower/Intrinsics/maskl.f90
    M flang/test/Lower/Intrinsics/maskr.f90

  Log Message:
  -----------
  [flang] Fix MASKR/MASKL lowering for INTEGER(16) (#87496)

The all one masks was not properly created for i128 types because
builder.createIntegerConstant ended-up truncating -1 to something
positive.

Add a builder.createAllOnesInteger/createMinusOneInteger helpers and use
them where createIntegerConstant(..., -1) was used.
Add an assert in createIntegerConstant to catch negative numbers for
i128 type.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications